Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add service area cluster to rvc device type template #34075

Conversation

hicklin
Copy link
Contributor

@hicklin hicklin commented Jun 25, 2024

Fixes #33364.

Adds the Service Area cluster to the RVC device type definition.

This PR depends on PR #33752 and #33991.

johnfierke and others added 30 commits May 22, 2024 11:42
…definition-for-service-area-cluster

    Merge remote-tracking branch 'origin/Add-common-definitions-for-Home-Location' into Add-xml-definition-for-service-area-cluster
    Pull in updates to Home-Location definitions from other PR branch.
… 6bf3762eb1ee733c642e79074744d1185b82a89c 2024-05-24

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
Copy link

github-actions bot commented Jun 26, 2024

PR #34075: Size comparison from ce71ed5 to 0e23d2a

Full report (85 builds for bl602, bl702, bl702l, cc13x4_26x4, cc32xx, cyw30739, efr32, esp32, linux, mbed, nrfconnect, nxp, psoc6, qpg, stm32, telink, tizen)
platform target config section ce71ed5 0e23d2a change % change
bl602 lighting-app bl602 FLASH 1270548 1270548 0 0.0
RAM 95328 95328 0 0.0
bl602+mfd FLASH 1284806 1284806 0 0.0
RAM 95472 95472 0 0.0
bl602+rpc FLASH 1309764 1309764 0 0.0
RAM 103752 103752 0 0.0
bl702 lighting-app bl702 FLASH 1091444 1091444 0 0.0
RAM 15161 15161 0 0.0
bl702+mfd FLASH 1102138 1102138 0 0.0
RAM 15313 15313 0 0.0
bl702+rpc FLASH 1181254 1181254 0 0.0
RAM 24181 24181 0 0.0
bl706-eth FLASH 874788 874788 0 0.0
RAM 27272 27272 0 0.0
bl706-wifi FLASH 1127088 1127088 0 0.0
RAM 14605 14605 0 0.0
bl702l lighting-app bl702l FLASH 1078330 1078330 0 0.0
RAM 21732 21732 0 0.0
bl702l+mfd FLASH 1089592 1089592 0 0.0
RAM 21892 21892 0 0.0
cc13x4_26x4 lighting-app LP_EM_CC1354P10_6 FLASH 798008 798008 0 0.0
RAM 103096 103096 0 0.0
lock-ftd LP_EM_CC1354P10_6 FLASH 813764 813764 0 0.0
RAM 113568 113568 0 0.0
lock-mtd LP_EM_CC1354P10_6 FLASH 803288 803288 0 0.0
RAM 107696 107696 0 0.0
pump-app LP_EM_CC1354P10_6 FLASH 756080 756080 0 0.0
RAM 101788 101788 0 0.0
pump-controller-app LP_EM_CC1354P10_6 FLASH 741752 741752 0 0.0
RAM 102036 102036 0 0.0
cc32xx air-purifier CC3235SF_LAUNCHXL FLASH 605754 605754 0 0.0
RAM 204508 204508 0 0.0
lock CC3235SF_LAUNCHXL FLASH 650782 650782 0 0.0
RAM 204780 204780 0 0.0
cyw30739 light CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 665785 665785 0 0.0
RAM 75100 75100 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 685637 685637 0 0.0
RAM 77732 77732 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 685637 685637 0 0.0
RAM 77732 77732 0 0.0
CYW930739M2EVB-02 unknown 2040 2040 0 0.0
FLASH 642565 642565 0 0.0
RAM 70168 70168 0 0.0
light-switch CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 608345 608345 0 0.0
RAM 70804 70804 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 627981 627981 0 0.0
RAM 73356 73356 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 627981 627981 0 0.0
RAM 73356 73356 0 0.0
lock CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 623825 623825 0 0.0
RAM 73820 73820 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 643541 643541 0 0.0
RAM 76372 76372 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 643541 643541 0 0.0
RAM 76372 76372 0 0.0
thermostat CYW30739B2-P5-EVK-01 unknown 2040 2040 0 0.0
FLASH 592397 592397 0 0.0
RAM 67788 67788 0 0.0
CYW30739B2-P5-EVK-02 unknown 2040 2040 0 0.0
FLASH 612249 612249 0 0.0
RAM 70420 70420 0 0.0
CYW30739B2-P5-EVK-03 unknown 2040 2040 0 0.0
FLASH 612249 612249 0 0.0
RAM 70420 70420 0 0.0
efr32 lighting-app BRD4187C FLASH 908680 908672 -8 -0.0
RAM 132492 132492 0 0.0
lock-app BRD4338a FLASH 745336 745328 -8 -0.0
RAM 172300 172300 0 0.0
window-app BRD4187C FLASH 995732 995732 0 0.0
RAM 127108 127108 0 0.0
esp32 all-clusters-app c3devkit DRAM 88332 88332 0 0.0
FLASH 1468676 1468676 0 0.0
IRAM 75570 75570 0 0.0
m5stack DRAM 114796 114796 0 0.0
FLASH 1537827 1537827 0 0.0
IRAM 125403 125403 0 0.0
linux air-purifier-app debug unknown 4560 4560 0 0.0
FLASH 2528400 2528400 0 0.0
RAM 125048 125048 0 0.0
all-clusters-app debug unknown 5344 5344 0 0.0
FLASH 5580286 5580286 0 0.0
RAM 487288 487288 0 0.0
all-clusters-minimal-app debug unknown 5264 5264 0 0.0
FLASH 5050568 5050568 0 0.0
RAM 232872 232872 0 0.0
bridge-app debug unknown 5232 5232 0 0.0
FLASH 4476584 4476584 0 0.0
RAM 212768 212768 0 0.0
chip-tool debug unknown 5696 5696 0 0.0
FLASH 11674855 11674855 0 0.0
RAM 541466 541466 0 0.0
chip-tool-ipv6only arm64 unknown 19944 19944 0 0.0
FLASH 10795796 10795796 0 0.0
RAM 590368 590368 0 0.0
fabric-admin debug unknown 5576 5576 0 0.0
FLASH 10641399 10641399 0 0.0
RAM 535514 535514 0 0.0
fabric-bridge-app debug unknown 5240 5240 0 0.0
FLASH 4347112 4347112 0 0.0
RAM 204896 204896 0 0.0
lighting-app debug+rpc+ui unknown 5904 5904 0 0.0
FLASH 5372258 5372258 0 0.0
RAM 221576 221576 0 0.0
lock-app debug unknown 5168 5168 0 0.0
FLASH 4540184 4540184 0 0.0
RAM 200184 200184 0 0.0
ota-provider-app debug unknown 4552 4552 0 0.0
FLASH 4195912 4195912 0 0.0
RAM 194448 194448 0 0.0
ota-requestor-app debug unknown 4488 4488 0 0.0
FLASH 4321400 4321400 0 0.0
RAM 199104 199104 0 0.0
shell debug unknown 4112 4112 0 0.0
FLASH 2794013 2794013 0 0.0
RAM 150416 150416 0 0.0
thermostat-no-ble arm64 unknown 9184 9184 0 0.0
FLASH 4167116 4167116 0 0.0
RAM 235800 235800 0 0.0
tv-app debug unknown 5448 5448 0 0.0
FLASH 5593304 5593304 0 0.0
RAM 341672 341672 0 0.0
tv-casting-app debug unknown 5056 5056 0 0.0
FLASH 9858798 9858798 0 0.0
RAM 399968 399968 0 0.0
mbed lock-app-release cy8cproto_062_4343w FLASH 1502076 1502076 0 0.0
RAM 226656 226656 0 0.0
nrfconnect all-clusters-app nrf52840dk_nrf52840 FLASH 880800 880800 0 0.0
RAM 139693 139693 0 0.0
nrf7002dk_nrf5340_cpuapp FLASH 951612 951612 0 0.0
RAM 138121 138121 0 0.0
all-clusters-minimal-app nrf52840dk_nrf52840 FLASH 826356 826356 0 0.0
RAM 138591 138591 0 0.0
nxp contact k32w0+release FLASH 575596 575596 0 0.0
RAM 70024 70024 0 0.0
k32w1+release FLASH 590736 590736 0 0.0
RAM 74056 74056 0 0.0
light k32w0+release FLASH 609680 609680 0 0.0
RAM 69500 69500 0 0.0
k32w1+release FLASH 674248 674248 0 0.0
RAM 82816 82816 0 0.0
psoc6 all-clusters cy8ckit_062s2_43012 FLASH 1613356 1613356 0 0.0
RAM 207148 207148 0 0.0
all-clusters-minimal cy8ckit_062s2_43012 FLASH 1534108 1534108 0 0.0
RAM 204052 204052 0 0.0
light cy8ckit_062s2_43012 FLASH 1461004 1461004 0 0.0
RAM 197332 197332 0 0.0
lock cy8ckit_062s2_43012 FLASH 1462860 1462860 0 0.0
RAM 224396 224396 0 0.0
qpg lighting-app qpg6105+debug FLASH 650612 650612 0 0.0
RAM 104564 104564 0 0.0
lock-app qpg6105+debug FLASH 610736 610736 0 0.0
RAM 99240 99240 0 0.0
stm32 light STM32WB5MM-DK FLASH 472056 472056 0 0.0
RAM 141652 141652 0 0.0
telink air-quality-sensor-app tlsr9528a_retention FLASH 632136 632136 0 0.0
RAM 50528 50528 0 0.0
all-clusters-app tlsr9118bdk40d FLASH 657028 657028 0 0.0
RAM 145872 145872 0 0.0
all-clusters-minimal-app tlsr9528a FLASH 777102 777102 0 0.0
RAM 110684 110684 0 0.0
bridge-app tlsr9258a FLASH 675120 675120 0 0.0
RAM 95304 95304 0 0.0
contact-sensor-app tlsr9528a_retention FLASH 633720 633720 0 0.0
RAM 50572 50572 0 0.0
light-switch-app-ota-shell-factory-data tlsr9528a FLASH 719514 719514 0 0.0
RAM 77148 77148 0 0.0
lighting-app-ota-factory-data tlsr9118bdk40d FLASH 612162 612162 0 0.0
RAM 142104 142104 0 0.0
lighting-app-ota-rpc-factory-data-4mb tlsr9518adk80d FLASH 799722 799722 0 0.0
RAM 100508 100508 0 0.0
lock-app-dfu tlsr9528a FLASH 665284 665284 0 0.0
RAM 69860 69860 0 0.0
ota-requestor-app tlsr9258a FLASH 694450 694450 0 0.0
RAM 95028 95028 0 0.0
pump-app tlsr9518adk80d FLASH 615984 615984 0 0.0
RAM 56952 56952 0 0.0
pump-controller-app tlsr9518adk80d FLASH 606368 606368 0 0.0
RAM 56752 56752 0 0.0
shell tlsr9518adk80d FLASH 466182 466182 0 0.0
RAM 72484 72484 0 0.0
smoke_co_alarm-app tlsr9528a_retention FLASH 640080 640080 0 0.0
RAM 52200 52200 0 0.0
temperature-measurement-app-mars-ota tlsr9518adk80d FLASH 650194 650194 0 0.0
RAM 60388 60388 0 0.0
thermostat tlsr9518adk80d FLASH 625258 625258 0 0.0
RAM 57084 57084 0 0.0
window-covering tlsr9118bdk40d FLASH 518710 518710 0 0.0
RAM 97800 97800 0 0.0
tizen all-clusters-app arm unknown 1568 1568 0 0.0
FLASH 1632944 1632944 0 0.0
RAM 46012 46012 0 0.0
chip-tool-ubsan arm unknown 2368 2368 0 0.0
FLASH 16081330 16081330 0 0.0
RAM 7065308 7065308 0 0.0

@mergify mergify bot merged commit 4f0b7e9 into project-chip:master Jun 26, 2024
70 checks passed
austina-csa pushed a commit to austina-csa/connectedhomeip that referenced this pull request Jul 10, 2024
)

* Add definitions for common HomeLocationStruct and types - see connectedhomeip-spec PR 8933

* Add definition for Service Area Cluster - see connectedhomeip-spec PR 8937

* Add semantic tag definitions needed for Service Area Cluster - see connectedhomeip-spec PR 8937

* Update Service Area Cluster definition of LocationInfoStruct - see connectedhomeip-spec PR 8937

* Rename SurfaceTag to FloorSurfaceTag - spec PR 8937 commit 431739b3e3996d0ef74a36b2a6b93ac0d3db9c45

* Fix Carport typo, remove IsGroundFloor field (spec change)

* Update home location definitions per spec PR 8933 commit f04958166412d5b7eff4d3443273f47f12f22baf 2024-05-23

* update generated files

* Update Service Area Cluster definitions per spec PR 8937 up to commit 6bf3762eb1ee733c642e79074744d1185b82a89c 2024-05-24

* update generated files

* Regenerated zap files after merge.

* Moved the AreaTypeTag namespace definition to namespaces.xml.

* Moved the HomeLoc global struct into global-sturcts.xml.

* Updated the AreaTypeTag namespace to match the latest spec definition.

* Updated the AreaTypeTag namespace to match the latest spec definition.

* Updated the matterlint rules with the changes in the xml files.

* Reverted changes to the rvc zap file.

* Addded global xml files to relevant lists of xmls.

* Added the Position, Landmark and Floor Surface Namespaces.

* Removed namespace tag definitions from the service area cluters XML.

* Regenerated zap files.

* Explicitly set the conformance.

* Fixed typos in the service-area clusters XML from review.

* Used a more feature rich method for defining the cluster's features.

* Reordered data-type definitions to match the order in the spec.

* Regenerated zap files.

* Added github issue to a comment.

* Renamed namespaces.xml to semantic-tag-namespace-enums.xml.

* Missed some rename locations.

* Added the Service Area cluster as an optional cluster to the Robotic Vacuun Cleaner device type.

* Regenerated zap files after sync with upstream.

* Regenerated zap files after sync with upstream.

---------

Co-authored-by: jfierke@irobot.com <jfierke@irobot.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Update the RVC device type definition
4 participants